Skip to content

domoticz: disable LTO (for breaking the build)#29239

Open
ryjelsum wants to merge 1 commit intoopenwrt:masterfrom
ryjelsum:domoticz-fix
Open

domoticz: disable LTO (for breaking the build)#29239
ryjelsum wants to merge 1 commit intoopenwrt:masterfrom
ryjelsum:domoticz-fix

Conversation

@ryjelsum
Copy link
Copy Markdown

📦 Package Details

Maintainer: @dwmw2

Description:
Sometime in Dec 2025, domoticz's package started to fail to build after a dependency update. The root of the problem is LTO, seemingly in combination with usage of -D_FORTIFY_SOURCE=1. LTO doesn't seem to be actively tested in domoticz's development anymore, so it is probably best to disable it. (Unless someone else has a better suggestion!)
See #29004


🧪 Run Testing Details

  • OpenWrt Version: 25.12.2
  • OpenWrt Target/Subtarget: ipq40xx/chromium
  • OpenWrt Device: Google Wifi (gale)
    (Compiled into an image and run on a device, and checked for basic functionality.)

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

@commodo
Copy link
Copy Markdown
Contributor

commodo commented Apr 26, 2026

Formalities need fix

Sometime in Dec 2025, domoticz's package started to fail to build after a
dependency update.The root of the problem is LTO, seemingly in combination
with usage of -D_FORTIFY_SOURCE=1.
LTO doesn't seem to be actively tested in domoticz's development anymore,
so it is probably best to disable it.

See openwrt#29004

Signed-off-by: Rudi Knauss <ryjelsum@gmail.com>
@ryjelsum ryjelsum changed the title domoticz: Disable LTO (for breaking the build) domoticz: disable LTO (for breaking the build) Apr 26, 2026
@ryjelsum
Copy link
Copy Markdown
Author

I think that force-push should fix the commit message formality issues? Let me know if there's anything else I need to fix up.

@hnyman
Copy link
Copy Markdown
Contributor

hnyman commented Apr 27, 2026

You could bump also the PKG_RELEASE
PKG_RELEASE:=4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants